fix(llm_flows): injecting TransferToAgentTool after request confirmation#5669
fix(llm_flows): injecting TransferToAgentTool after request confirmation#5669settler-av wants to merge 3 commits into
Conversation
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
e161b73 to
cb0dd31
Compare
|
Hi @settler-av , Thank you for your contribution through this pull request! This PR has merge conflicts that require changes from your end. Could you please rebase your branch with the latest main branch to address these? Once this is complete, please let us know so we can proceed with the review. |
dadf88f to
80098d7
Compare
|
Hi @rohityan, Thanks for reviewing the PR! I have resolved the merge conflict with main branch. We can proceed further with the review. |
fix(flows): inject transfer_to_agent tool for HITL confirmation resume
Closes: #5633
Problem:
When using HITL (human-in-the-loop) confirmation for
transfer_to_agent, the tool was not present intools_dictduring confirmation resume, causing a ValueError and breaking the transfer flow.Solution:
transfer_to_agentis injected intotools_dictduring confirmation resume, mirroring the logic in the agent transfer processor.Testing Plan:
pytest test_request_confirmation.py -vChecklist:
Additional context:
No API or breaking changes. Fix is isolated to HITL confirmation resume logic.